feat (suggestion): add shared skills settings#1085
feat (suggestion): add shared skills settings#1085JustYannicc wants to merge 2 commits intopingdotgg:mainfrom
Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
📝 Coding Plan
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment Tip You can make CodeRabbit's review stricter and more nitpicky using the `assertive` profile, if that's what you prefer.Change the |
github-actions
bot
added
size:XXL
![macroscopeapp[bot]](https://avatars.githubusercontent.com/in/900172?s=60&v=4){kind=small}
JustYannicc
force-pushed
the
codex/issue-1081-shared-skills
branch
from
a78c30d to
1120ea8
Compare
1 participant
What Changed
Implements shared skill management #1081.
This adds a
Skillssection in Settings, similar to Codex, where shared skills can be initialized and managed. On first use, skills are moved into a user-defined shared directory and symlinked back into the harness skill roots so Codex can keep working as usual without needing to know anything about the shared setup.The PR also adds:
Why
The goal is to make local skills more portable across harnesses while keeping Codex behavior unchanged.
I mainly built this for myself so I can keep skills in one shared location and eventually reuse the same setup across tools like Codex and Claude Code. From Codex’s perspective, it should still just see normal skill directories in the places it already expects, because the shared setup is handled through symlinks.
This is tied to #1081, but there is no expectation that this has to be merged. I’m opening it in case the implementation is useful, and I’m completely fine if you decide not to take it.
If there are any changes I have to make, let me know.
UI Changes
Settings Screen
Before:
After:
New Skills Setting Screen
Single Skill clicked in
Checklist
Note
Add shared skills settings panel with lifecycle management via WebSocket API
SkillsSettingsPanelcomponent under a new "Skills" tab in the settings page, reachable via?tab=skillsin the URL.sharedSkills.tson the server with functions to discover, migrate, enable/disable, and uninstall skills across Codex and Agents harness directories using a shared directory with symlinks.getSharedSkills,getSharedSkillDetail,initializeSharedSkills,setSharedSkillEnabled,uninstallSharedSkill) backed by corresponding React Query hooks insharedSkillsReactQuery.ts.sharedSkillsPathtoAppSettingsSchemawith an empty-string default, allowing users to configure the shared skills directory via folder picker in the UI.packages/contractsfor all shared skill operations.📊 Macroscope summarized a78c30d. 12 files reviewed, 3 issues evaluated, 0 issues filtered, 3 comments posted
🗂️ Filtered Issues